Skip to content

Convert to vite native tooling#2118

Open
ejsmith wants to merge 1 commit intomainfrom
vite-native
Open

Convert to vite native tooling#2118
ejsmith wants to merge 1 commit intomainfrom
vite-native

Conversation

@ejsmith
Copy link
Member

@ejsmith ejsmith commented Feb 17, 2026

This pull request introduces Oxlint and Oxfmt into the project's frontend workflow to improve code quality and formatting consistency. It updates configuration files, scripts, and dependencies to integrate these tools alongside existing linting and formatting solutions. The changes also adjust the ESLint and Vite configurations to support Oxlint, and update the package.json to use the latest rolldown-vite build.

Tooling and Linting Integration:

  • Added .oxlintrc.json and .oxfmtrc.json configuration files to define project-specific rules for Oxlint and Oxfmt. [1] [2]
  • Updated eslint.config.js to include eslint-plugin-oxlint and to load Oxlint configuration, ensuring ESLint rules do not overlap with Oxlint. [1] [2]
  • Integrated vite-plugin-oxlint into the Vite configuration, enabling Oxlint checks during the build process. [1] [2]

Scripts and Dependency Updates:

  • Modified package.json scripts to add lint:oxfmt, lint:oxlint, and corresponding format commands, and to ensure Oxlint and Oxfmt are run as part of the main linting and formatting workflows.
  • Added eslint-plugin-oxlint, vite-plugin-oxlint, oxlint, and oxfmt to devDependencies.
  • Switched the vite dependency to use the latest rolldown-vite build and updated the overrides section accordingly.

These changes help enforce stricter code standards and ensure a more consistent codebase by leveraging the latest linting and formatting tools.

@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Complexity Health
Exceptionless.Insulation 24% 23% 208
Exceptionless.Core 66% 59% 7446
Exceptionless.AppHost 26% 14% 55
Exceptionless.Web 55% 42% 3489
Summary 61% (11865 / 19532) 53% (5655 / 10706) 11198

@ejsmith
Copy link
Member Author

ejsmith commented Mar 16, 2026

@copilot update this PR now that vite 8 final is out. Update any deps to latest versions. Get the build working.

Copy link
Contributor

Copilot AI commented Mar 16, 2026

@ejsmith I've opened a new pull request, #2152, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants